Skip to content

Add --no-aosp flag to select Google Style - #1415

Open
hferreiro wants to merge 1 commit into
google:masterfrom
hferreiro:no-aosp
Open

Add --no-aosp flag to select Google Style#1415
hferreiro wants to merge 1 commit into
google:masterfrom
hferreiro:no-aosp

Conversation

@hferreiro

Copy link
Copy Markdown

--aosp is the only style switch, with no way to negate it once passed. Wrapper scripts such as Chromium's google-java-format launcher append --aosp unconditionally, leaving projects that format through them no way to select Google Style. Add a --no-aosp flag to allow overriding a previously given style flag. When both are given, the last one wins.

--aosp is the only style switch, with no way to negate it once passed.
Wrapper scripts such as Chromium's google-java-format launcher append --aosp
unconditionally, leaving projects that format through them no way to select
Google Style. Add a --no-aosp flag to allow overriding a previously given style
flag. When both are given, the last one wins.
@cushon

cushon commented Aug 25, 2026

Copy link
Copy Markdown
Collaborator

Thanks, this looks fine.

One minor question: WDYT about --google-style or something like that instead of --no-aosp? I don't think we're expecting to add more styles here, and the behaviour would be the same, I just wonder if it's slightly clearer.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants